-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Auto-label PRs by outside collaborators #648
Auto-label PRs by outside collaborators #648
Conversation
@lamdbui it would be good to hear what we want this to look like—whether it should be auto-assignment, a label automatically applied to the PR, or something else entirely |
See an example here: youtube/cobalt_sandbox#467 |
47d9ae7
to
23b80db
Compare
Codecov Report
@@ Coverage Diff @@
## main #648 +/- ##
==========================================
- Coverage 56.51% 56.50% -0.02%
==========================================
Files 1897 1897
Lines 94217 94217
==========================================
- Hits 53247 53236 -11
- Misses 40970 40981 +11 |
I'd start with a label, as it is now. Easy to make a go-link that just shows pending ones. |
Quick ping on this - seems like a good thing to land. @lamdbui |
Sorry it took a while for me to get to this. Yes, a label sounds great. As long as we have an on-going process for review with say a go/ link as @kaidokert mentioned, we can track this. If we find that we need an alias or something, we can add it later, but let's keep it simple for now. |
b/261052420